Skip to content
This repository has been archived by the owner on Sep 2, 2023. It is now read-only.

doc: add meeting notes for 2018-03-28 #53

Merged
merged 4 commits into from
Apr 25, 2018
Merged

doc: add meeting notes for 2018-03-28 #53

merged 4 commits into from
Apr 25, 2018

Conversation

MylesBorins
Copy link
Contributor

No description provided.

TimothyGu

This comment was marked as off-topic.

Sign in to view

ljharb

This comment was marked as off-topic.

Sign in to view

inidaname

This comment was marked as off-topic.

Sign in to view

@robpalme
Copy link
Contributor

robpalme commented Mar 29, 2018 •
edited
Loading

At the end, in the discussion of interop and require()'s ability to synchronously load ESM, someone (I'm not sure who) stated that Top-Level Await will prevent this. @jdalton responded that @std/esm already solves this, "what can I tell you".

Having checked @std/esm I see it only implements TLA for modules without the export keyword. So it seems like the original point stands.

@jdalton
Copy link
Member

jdalton commented Mar 29, 2018 •
edited
Loading

Having checked @std/esm I see it only implements TLA for modules without the export keyword. So it seems like the original point stands.

To be clear I support variant C of the stage-1 proposal. It seems to do the trick 👌

@robpalme
Copy link
Contributor

Ok - thank you for clarifying.

@MylesBorins MylesBorins added the modules-agenda To be discussed in a meeting label Apr 2, 2018
mhdawson

This comment was marked as off-topic.

Sign in to view

@MylesBorins MylesBorins mentioned this pull request Apr 10, 2018
Closed
mcollina

This comment was marked as off-topic.

Sign in to view

jdalton

This comment was marked as off-topic.

Sign in to view

@MylesBorins MylesBorins mentioned this pull request Apr 23, 2018
Closed
manekinekko

This comment was marked as off-topic.

Sign in to view

linclark

This comment was marked as off-topic.

Sign in to view

@devsnek devsnek merged commit 55ba1cd into nodejs:master Apr 25, 2018
@GeoffreyBooth GeoffreyBooth removed modules-agenda To be discussed in a meeting labels May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ljharb ljharb ljharb left review comments

@mhdawson mhdawson mhdawson approved these changes

@inidaname inidaname inidaname approved these changes

@manekinekko manekinekko manekinekko approved these changes

@TimothyGu TimothyGu TimothyGu approved these changes

@linclark linclark linclark approved these changes

@mcollina mcollina mcollina approved these changes

@jdalton jdalton jdalton approved these changes

Assignees
No one assigned
Labels
doc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.